Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

MR: Merge Worker issue #57 to main #63

Closed
wants to merge 22 commits into from
Closed

Conversation

mr0321
Copy link
Contributor

@mr0321 mr0321 commented Apr 19, 2024

Possible implementation for #57
Can do:
Get Context from task message from RabbitMQ: gets data, merges models based on properties fields, add context to logger (elastic logs)
Get Data from Object Storage: gets data from local storage (elastic, minio) fallback can be set to opdm (options to run the validation or model retriever pipeline)
Publish Data to Object Storage: publishes data to local storage, file tree (prefix) is composed from the filename. Metadata can be published to elastic
Cannot do:
Publish Data to OPDM: publishes the data to opdm but from the opdm side, validation still fails

mr0321 added 21 commits March 7, 2024 14:53
For using look examples at the end of validator.py
…somewhat faster than printing the log to console (validator.py line 706)
…e files in directory, parses them to the form as they would have come from OPDE and continues as usual. Plus some bug fixes for running it externally
…troduces two handlers: HandlerGetModels (gets data from minio and feeds to cgm_compose, HandlerMergeModels calls the merge and posts the results
@mr0321 mr0321 marked this pull request as ready for review April 19, 2024 08:34
@Haigutus Haigutus changed the title MR: issue #57 to main MR: Merge Worker issue #57 to main May 13, 2024
@mr0321 mr0321 closed this May 16, 2024
@mr0321 mr0321 deleted the mr-57-create-merge-worker branch May 22, 2024 07:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant